Skip to content

fix(messages): restore main typecheck for pasted mentions - #684

Merged
wesbillman merged 1 commit into
mainfrom
larry/fix-main-mention-candidates
Oct 6, 2026
Merged

wesbillman merged 1 commit into
mainfrom
larry/fix-main-mention-candidates

Conversation

@loganj

@loganj loganj commented Oct 6, 2026

Copy link
Copy Markdown
Contributor

🤖

Summary

  • Main does not build. Two pull requests merged within minutes of each other and each was correct alone: feat(messages): copy and paste mentions, channel refs and links #579 (paste mentions) checked pasted identity links with a helper that refactor(mentions): let the Mentions plugin own what the chooser offers #669 (Mentions plugin owns the chooser) moved out of the app shell. TypeScript now fails in MessageComposer.tsx.
  • This change restores the build without changing what users see. When you paste a link to a person, the composer still notifies them under their current profile name, never the pasted label. A key with no known name stays plain text unless that person is in the channel. Edits still never add recipients.

Details

#579 added pasted identity links using mentionCandidates, which #669
moved into the Mentions plugin. Main no longer typechecks. The host now
names a pasted recipient through mention-admission, keeping #579's
behaviour: current names only, and an unknown outsider stays display-only.

Signed-off-by: Larry <627498bd4bd1f281a16431e3c6cce3b5c25b6692798c78672298aefbf2f8f8b5@buzz.block.builderlab.xyz>
@wesbillman
wesbillman marked this pull request as ready for review October 6, 2026 23:54
@wesbillman
wesbillman requested review from a team, comp615 and wesbillman as code owners October 6, 2026 23:54
@wesbillman
wesbillman merged commit b03be61 into main Oct 6, 2026
22 checks passed
@wesbillman
wesbillman deleted the larry/fix-main-mention-candidates branch October 6, 2026 23:55
johnmatthewtennant pushed a commit that referenced this pull request Oct 7, 2026
…ge-delete

* origin/main: (23 commits)
  feat(agents): default conversation context to each thread (#646)
  Name browser host and separate development tooling (#699)
  test(archive): stamp expiry fixtures with one fixed time (#697)
  feat(messages): list Mark unread above Copy message (#638)
  fix(messages): link credential-free http:// URLs like https:// (#691)
  refactor(hooks): replace custom wiring with standard Lefthook hooks (#641)
  fix: deduplicate working agent indicators in composers (#687)
  feat(messages): click images to zoom in the gallery viewer (#688)
  fix(media): serve ranged buzz-media reads from shared signed blocks (#690)
  fix(messages): exclude row chrome from cross-message copies (#645)
  docs: Add kind 30177 publishing guidance for plugin authors (#558)
  feat(agents): allow creating agents with Claude Code
  feat: Enroll Builderlab remote agents (30177) to community (#647)
  Enroll Builderlab agents with the selected community during attestation (#642)
  fix(messages): restore main typecheck for pasted mentions (#684)
  Install the bundled Buzz CLI skill on desktop startup (#667)
  refactor: share status, DM and sidebar intent policy across hosts (#683)
  fix(sessions): show enabled page in navigation (#648)
  fix(composer): replace exact emoji shortcodes on closing colon (#584)
  refactor(mentions): let the Mentions plugin own what the chooser offers (#669)
  ...

Signed-off-by: Opus <f425b86c3b780d43f8d9cf1802429d04dd985bf92eac44fca9468b5ad3887c5e@buzz.block.builderlab.xyz>

# Conflicts:
#	src/features/messages/MessageManagement.tsx
johnmatthewtennant pushed a commit that referenced this pull request Oct 7, 2026
* origin/main:
  feat: Enroll Builderlab remote agents (30177) to community (#647)
  Enroll Builderlab agents with the selected community during attestation (#642)
  fix(messages): restore main typecheck for pasted mentions (#684)

Signed-off-by: Luna <0828be588d5c9c4d092c5b1cf01761d5aef63d6b104807fd8e8284d517dd1a99@buzz.block.builderlab.xyz>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants